-
Notifications
You must be signed in to change notification settings - Fork 159
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Action toasts #5147
Action toasts #5147
Conversation
💥 Acceptance tests webUINotificationBasic failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15655/
webUINotifications-displayNotificationsOnWebUI-feature-18.png |
💥 Acceptance tests TrashbinDelete failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15655/
webUITrashbinDelete-trashbinDelete-feature-113.pngwebUITrashbinDelete-trashbinDelete-feature-29.pngwebUITrashbinDelete-trashbinDelete-feature-42.png |
💥 Acceptance tests MarkdownEditor failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15655/
webUIMarkdownEditor-markdownFile-feature-62.pngwebUIMarkdownEditor-markdownFile-feature-63.png |
💥 Visual regression tests webUIOCISFiles3 failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15655/
|
💥 Acceptance tests TrashbinFilesFolders failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15655/
webUITrashbinFilesFolders-trashbinFilesFolders-feature-23.pngwebUITrashbinFilesFolders-trashbinFilesFolders-feature-38.pngwebUITrashbinFilesFolders-trashbinFilesFolders-feature-82.pngwebUITrashbinFilesFolders-trashbinFilesFolders-feature-97.png |
💥 Acceptance tests SharingInternalUsersExpireToRoot failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15655/
webUISharingInternalUsersExpireToRoot-shareWithUsersExpiringShares-feature-232.png |
💥 Acceptance tests SharingInternalUsersExpire failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15655/
webUISharingInternalUsersExpire-shareWithUsersExpiringShares-feature-240.png |
💥 Acceptance tests SharingPublicExpire failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15655/
webUISharingPublicExpire-shareByPublicLinkExpiringLinks-feature-106.pngwebUISharingPublicExpire-shareByPublicLinkExpiringLinks-feature-11.pngwebUISharingPublicExpire-shareByPublicLinkExpiringLinks-feature-147.pngwebUISharingPublicExpire-shareByPublicLinkExpiringLinks-feature-24.pngwebUISharingPublicExpire-shareByPublicLinkExpiringLinks-feature-86.png |
💥 Acceptance tests SharingPublicDifferentRoles failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15655/
webUISharingPublicDifferentRoles-shareByPublicLinkDifferentRoles-feature-114.pngwebUISharingPublicDifferentRoles-shareByPublicLinkDifferentRoles-feature-93.png |
💥 Acceptance tests oc10-integration-notifications failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15655/
webUISharingNotifications-shareWithUsers-feature-21.pngwebUISharingNotifications-shareWithUsers-feature-32.pngwebUISharingNotifications-shareWithUsers-feature-40.pngwebUISharingNotifications-shareWithUsers-feature-53.pngwebUISharingNotificationsToRoot-shareWithUsers-feature-19.pngwebUISharingNotificationsToRoot-shareWithUsers-feature-31.pngwebUISharingNotificationsToRoot-shareWithUsers-feature-40.pngwebUISharingNotificationsToRoot-shareWithUsers-feature-53.png |
💥 Acceptance tests Upload failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/web/15655/
webUIUpload-upload-feature-130.pngwebUIUpload-upload-feature-151.pngwebUIUpload-upload-feature-89.pngwebUIUpload-upload-feature-98.pngwebUIUpload-uploadEdgecases-feature-109.png |
Also fixes #3588 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 📬
Description
This PR introduces confirmation toasts for copying (private/public) links to the clipboard. Refactored into a
CopyToClipboardButton
component. We can decide later on if we want to move it to ODS (would need to extract the confirmation message then, as that needs to stay in web, and replace it with just emitting an event).Motivation and Context
a11y improvements
How Has This Been Tested?
Types of changes
Checklist: